-
Notifications
You must be signed in to change notification settings - Fork 149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
systemd user service display support and manual setup docs #107
base: master
Are you sure you want to change the base?
systemd user service display support and manual setup docs #107
Conversation
Hi, But I feel more and more is unclear to me. For example: when should a user do manual systemd configuration vs letting this plugin auto configure things? The current docs state this: |
Oh, and also: I would appreciate help with maintaining linux/systemd stuff. |
@bruno- thanks for taking a look. You are of course right, the docs should be more clear about what to do. The autoconfiguration looks good, but this PR should be merged so that it works correctly. In my case, the systemd service file was not created, since I did not have the systemd user folder on my system. Hence, I was forced to manually configure the service (and the docs reflect this approach). I can work on this PR a bit more and try to help with the tmux / linux autoconfigure script. The manual approach could be converted to an explanation section. what do you think? Regarding your second comment: While I use your great plugin and tmux daily, I could maybe help, but I am not a tmux expert at all. Let's work on improving support for linux / systemd for the plugin for now and then we shall see, alright? |
@nuclearglow thanks for clarifying things. I have merged #106 based on your input.
I'm worried this becomes detailed documentation that duplicates the content from the actual scripts. For example: if the docs contain the systemd unit file, then whenever we make a change to I would rather focus on making sure our automated scripting approach works flawlessly. What do you think? |
What about
That works. Yes, I only need help with linux and systemd so let's focus on that. My main problem is I don't have a linux machine where I can test stuff. And I'm not sure a virtual server has the same environment as a regular laptop installation. Would you be ok if we contacted you for any change to these two files:
I was thinking we put the following header at the top of these files:
Let me know what you think? |
@bruno- alright, that should work. I will try to help! Regarding the DISPLAY variable, I am not a 100% sure. I know that my tmux systemd service works fine right now, on different Ubuntu systems with different gpus (AMD Vega, NVidia RTX, Intel built-in) and with different monitor configurations. But it might be that if we wait for the graphical target anyway, it might not be needed. I think I will need to test that. I'll update the PR after I checked that. |
Thanks. I have added you as a maintainer to systemd files via 9329afa
I propose you just open a new PR for DISPLAY variable, if you think a change is necessary. |
related: https://github.com/tmux-plugins/tmux-continuum/pull/106/files
related: https://github.com/tmux-plugins/tmux-continuum/pull/102/files